-
Notifications
You must be signed in to change notification settings - Fork 22.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(ci): Add Reviewdog for Prettier formatting #28228
Conversation
Co-authored-by: Brian Thomas Smith <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No objections from me, leaving my +1 👍🏻
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@@ -279,7 +279,7 @@ If you nest a block `@layer` at-rule inside another block `@layer` at-rule, with | |||
|
|||
Let's look at the following example: | |||
|
|||
``` | |||
```css |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: I am quite literally only including this change because there's an issue with the required test list that prevents infrastructure-only PRs from getting merged. This change is not related to adding Reviewdog, but we have to modify some file in files/
to unblock merging.
Preview URLs (comment last updated: 2023-07-29 06:03:48) |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This PR introduces reviewdog as a CI workflow to suggest formatting changes to match Prettier's desires, using the reviewdog/action-suggester workflow. This resolves https://github.com/orgs/mdn/discussions/428.